Skip to content

Comments

update ica_comparison.py#13211

Closed
Ganasekhar-gif wants to merge 10 commits intomne-tools:mainfrom
Ganasekhar-gif:ica_comparison-example
Closed

update ica_comparison.py#13211
Ganasekhar-gif wants to merge 10 commits intomne-tools:mainfrom
Ganasekhar-gif:ica_comparison-example

Conversation

@Ganasekhar-gif
Copy link

EOG Component Detection
Reference Issue:
Addresses #9002

What does this implement/fix?
This draft PR improves the ICA comparison example by:
Comparing multiple ICA algorithms: FastICA, Picard, Infomax, and Extended Infomax.
Evaluating performance on both clean and synthetically noisy MEG data.
Detecting and visualizing EOG-related components for each method.
Showing side-by-side component comparisons to enhance interpretability.
Displaying fit time per method for performance benchmarking.
Providing a more informative educational example (moving beyond just a "how-to").

Additional Information:
This is a work-in-progress PR as encouraged by the maintainers. I'm looking for early feedback on:
Visual clarity of EOG component comparisons.
Whether to include more deterministic checks (e.g., random_state).

In addition, I have a couple of questions:
EOG Component Comparison:
The issue refers to comparing EOG components. Currently, I've selected the most likely EOG-related component based on each ICA method's output. Could you clarify what “the same” EOG component refers to in this context? Are you expecting comparisons based on a specific pre-determined EOG component or should this focus on the component detected by each algorithm?

Summary of Changes:
Slightly reworded the description for clarity.
Structured the PR content with bullet points to improve readability.
Reworded the EOG component question to avoid confusion.

Ganasekhar-gif and others added 10 commits April 14, 2025 17:40
added a towncrier entry
make some small changes in 13211.newfeature.rst file
Add visualization of clean and noisy EOG ICA components
Add visualization of clean and noisy EOG ICA components
fixed circleCI check issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant